ARROW-15454: [Python] Try to make CSV cancellation test more robust#12260
ARROW-15454: [Python] Try to make CSV cancellation test more robust#12260pitrou wants to merge 1 commit into
Conversation
|
|
|
@github-actions crossbow submit -g python |
|
Revision: 12f1b96519fe388e81ca400c075b900b705617aa Submitted crossbow builds: ursacomputing/crossbow @ actions-1493
|
There was a problem hiding this comment.
This allows to run the cancellation tests on Windows.
|
Since it only occurred in the M1 wheel verification tests I think it's fine to keep the workaround for 7.0 and ship this with 8.0. |
|
@github-actions crossbow submit wheel-macos-big-sur-cp38-arm64 |
There was a problem hiding this comment.
Now the failure should occur unless the patch resolves it.
|
Revision: 9525671caa3d0bb5b2913733253783c1fe1525d2 Submitted crossbow builds: ursacomputing/crossbow @ actions-1495
|
|
@github-actions crossbow submit wheel-macos-big-sur-cp38-arm64 |
|
Revision: 9525671caa3d0bb5b2913733253783c1fe1525d2 Submitted crossbow builds: ursacomputing/crossbow @ actions-1496
|
kszucs
force-pushed
the
ARROW-15454-csv-cancellation
branch
from
9525671 to
3490d79
Compare
|
Looks like the changes here weren't enough to fix the test on macOS M1. |
Use `signal.set_wakeup_fd` to wait reliably for the signal to get to the Python interpreter.
pitrou
force-pushed
the
ARROW-15454-csv-cancellation
branch
from
3490d79 to
3b324e1
Compare
|
I've removed the Crossbow tasks changes and will merge if green. |
|
Note the Python MacOS build was cancelled because the dependency install phase took too long. |
|
Benchmark runs are scheduled for baseline = f9f6fdb and contender = 9e69ccf. 9e69ccf is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
4 participants
Use
signal.set_wakeup_fdto wait reliably for the signal to get to the Python interpreter.